-
Notifications
You must be signed in to change notification settings - Fork 115
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: auto field value update #729
Conversation
|
129e6c0
to
ad90606
Compare
Deploying conform with
|
Latest commit: |
dbddf13
|
Status: | ✅ Deploy successful! |
Preview URL: | https://04b663ad.conform.pages.dev |
Branch Preview URL: | https://auto-field-update.conform.pages.dev |
ad90606
to
5ab22b6
Compare
More templates
@conform-to/dom
@conform-to/react
@conform-to/yup
@conform-to/zod
@conform-to/validitystate
commit: |
5ab22b6
to
dbddf13
Compare
Fix #600
This is my 2nd attempt to remove the need of passing the
key
to each input manually by updating the fields value automatically. As mentioned in #728...However, the previous solution made an wrong assumption that people will always subscribe to the
initialValue
when setting up a field which conflicts with (2). As there seems to be no way to associate a field with a subscription, we will either need to re-run the same side effect on every subscription, or, have theuseForm()
hook subscribed to all key changes so it will always re-render and run the side effect in one single place.This PR uses the later approach.